cmd/bd: teach bd prime to use --claim#2541
Merged
steveyegge merged 1 commit intogastownhall:mainfrom Mar 12, 2026
Merged
Conversation
Contributor
Author
|
Closing this draft for now because I opened it before proving the repo-level test baseline on latest upstream. I’ll verify full-suite behavior on current origin/main first and only reopen if the validation story is actually clean. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
bd primeto teachbd update <id> --claimin the command reference and starting-work example--status=in_progressguidance does not come backWhy
bd update --help,README.md, and other command surfaces already teach atomic claim syntax, butcmd/bd/prime.gostill told users to use--status=in_progress.Related: #2007
Validation
go test -count=1 ./cmd/bdpassed 3 consecutive runs on cleanorigin/main(a5827cb2)go test -count=1 ./cmd/bdpassed 3 consecutive runs oncodex/bd-prime-atomic-claimgo test -count=1 ./cmd/bd/doctor ./internal/utilsmatches on both sides:cmd/bd/doctorpasses andinternal/utilsfails inTestResolvePartialID_Wispgo test ./cmd/bd -run 'TestOnboardCommand|TestOutputContextFunction|TestPrimeClaimGuidanceUsesAtomicClaim'golangci-lint run ./cmd/bd/...Repo-level test baseline
make testis currently red on latestorigin/main, and it is also red on this branch. The reruns above are included to show that this patch does not worsen the changed package and that the unchanged noisy packages line up with upstream baseline behavior.